Skip to content

chore(CI/CD): Prevent agents from running npm#8930

Open
lstein wants to merge 1 commit intomainfrom
lstein/cicd/prevent-agents-from-running-npm
Open

chore(CI/CD): Prevent agents from running npm#8930
lstein wants to merge 1 commit intomainfrom
lstein/cicd/prevent-agents-from-running-npm

Conversation

@lstein
Copy link
Collaborator

@lstein lstein commented Feb 28, 2026

Summary

Copilot, and presumably other coding agents, have a tendency to run npm when updating the front end code. This creates an unwanted package-lock.json file.

This PR does three things:

  1. It adds an AGENTS.md file that gives coding agents explicit instructions to use pnpm rather than npm.
  2. It adds a packageManager instruction to invokeai/frontend/web/scripts/package.json (which seemed to be missing)
  3. It adds a check to the frontendchecks.yml workflow that will generate an error if package-lock.json is found in the PR.

Related Issues / Discussions

QA Instructions

  1. Using a coding agent, ask it to perform a pnpm operation such as lint or build.
  2. Confirm that it uses the proper tool.
  3. Add a package-lock.json file to invokeai/frontend/web and try to push it into this PR. The frontend-checks check should fail.

Merge Plan

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • ❗Changes to a redux slice have a corresponding migration
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

@lstein lstein self-assigned this Feb 28, 2026
@github-actions github-actions bot added CI-CD Continuous integration / Continuous delivery frontend-deps PRs that change frontend dependencies frontend PRs that change frontend files labels Feb 28, 2026
@lstein lstein assigned Pfannkuchensack and unassigned lstein Feb 28, 2026
@lstein lstein changed the title chore(CI/CD): Prevent agents from runing npm chore(CI/CD): Prevent agents from running npm Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-CD Continuous integration / Continuous delivery frontend PRs that change frontend files frontend-deps PRs that change frontend dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants